Skip to content

Conversation

@llucax
Copy link
Contributor

@llucax llucax commented Jun 2, 2025

No description provided.

Copilot AI review requested due to automatic review settings June 2, 2025 11:11
@llucax llucax requested a review from a team as a code owner June 2, 2025 11:11
@llucax llucax requested a review from nhatcher-frequenz June 2, 2025 11:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR bumps the versions of protobuf, grpcio-tools, and grpcio and updates the dependency constraints accordingly.

  • Update protobuf version from 5.29.3 to 6.31.1 and adjust its range.
  • Update grpcio-tools and grpcio versions from 1.70.0 to 1.72.1 while maintaining the grpcio dependency range.
Comments suppressed due to low confidence (2)

pyproject.toml:39

  • Verify that increasing the upper bound for protobuf to < 8 aligns with the library’s backwards compatibility guarantees and document the rationale.
"protobuf >= 6.31.1, < 8", # Do not widen beyond 8!

pyproject.toml:43

  • Confirm that the dependency range for grpcio remains appropriate after the upgrade and consider providing contextual documentation if needed.
"grpcio >= 1.72.1, < 2", # Do not widen beyond 2!

@github-actions github-actions bot added the part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) label Jun 2, 2025
@llucax llucax enabled auto-merge June 2, 2025 11:35
@llucax llucax self-assigned this Jun 2, 2025
Signed-off-by: Leandro Lucarella <[email protected]>
@llucax
Copy link
Contributor Author

llucax commented Jun 5, 2025

Will force merge these, we are already taking it for other repos.

@llucax llucax disabled auto-merge June 5, 2025 07:19
@llucax llucax merged commit 258eeee into frequenz-floss:v0.x.x Jun 5, 2025
6 checks passed
@llucax llucax added this to the v0.7.0 milestone Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:tooling Affects the development tooling (CI, deployment, dependency management, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant